From 0d48234f73150ce2c52c6adcc54e75e13452835b Mon Sep 17 00:00:00 2001 From: "Carol (Nichols || Goulding)" Date: Sat, 19 Sep 2015 14:39:34 -0400 Subject: [PATCH] Remove unnecessary PathSource update in cargo test I don't think updating the source here is hurting anything, per se, it's just unnecessary since ops::compile makes a PathSource and then calls root_package, which will call update if it needs to. --- src/cargo/ops/cargo_test.rs | 7 ------- 1 file changed, 7 deletions(-) diff --git a/src/cargo/ops/cargo_test.rs b/src/cargo/ops/cargo_test.rs index 8b5ca97a5..555b239c0 100644 --- a/src/cargo/ops/cargo_test.rs +++ b/src/cargo/ops/cargo_test.rs @@ -1,8 +1,6 @@ use std::ffi::{OsString, OsStr}; use std::path::Path; -use core::Source; -use sources::PathSource; use ops::{self, ExecEngine, ProcessEngine, Compilation}; use util::{self, CargoResult, CargoTestError, ProcessError}; @@ -63,11 +61,6 @@ pub fn run_benches(manifest_path: &Path, fn compile_tests<'a>(manifest_path: &Path, options: &TestOptions<'a>) -> CargoResult> { - let config = options.compile_opts.config; - let mut source = try!(PathSource::for_path(&manifest_path.parent().unwrap(), - config)); - try!(source.update()); - let mut compilation = try!(ops::compile(manifest_path, &options.compile_opts)); compilation.tests.sort(); Ok(compilation) -- 2.30.2